Skip to content

fix: default action can't work for notification #936

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
deepin-bot merged 1 commit into from
Dec 6, 2024
Merged

fix: default action can't work for notification #936

deepin-bot merged 1 commit into from
Dec 6, 2024

Conversation

@18202781743
Copy link
Contributor

@18202781743 18202781743 commented Dec 6, 2024

Default action maybe have text, and it's actionId is default.
Remove entity when manually closing it before timeout.
CloseNotification can't work.

pms: BUG-290767

@deepin-ci-robot
Copy link

deepin-ci-robot commented Dec 6, 2024

deepin pr auto review

代码审查意见：

  1. 在BubbleItem::updateActions和AppNotifyItem::updateActions函数中，当移除"default"动作时，应该检查actions列表是否为空，以避免在后续操作中发生越界访问。

  2. 在NotifyEntity类中添加了operator!=，但是没有在operator==中处理operator!=的情况，可能会导致逻辑错误。

  3. 在NotificationManager::Notify函数中，当replacesId为0时，返回的是entity.bubbleId()，而不是entity.id()，这可能不符合预期。

  4. 在NotificationManager::CloseNotification函数中，如果通知不存在，应该发送一个错误消息，而不是静默失败。

  5. 在NotificationManager::actionInvoked函数中，doActionInvoked函数调用应该在entity.isValid()检查之后，以确保只有在有效实体上执行操作。

  6. 在NotificationManager::updateEntityProcessed函数中，removePendingEntity函数被调用，但是没有检查entity是否为有效实体。

  7. 在NotificationManager::removePendingEntity函数中，m_pendingTimeoutEntities是一个QHash，应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  8. 在NotificationManager::removePendingEntity函数中，onHandingPendingEntities函数被调用，但是没有检查m_pendingTimeoutEntities是否为空，可能会导致不必要的调用。

  9. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该返回一个布尔值，表示是否成功移除了实体。

  10. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该抛出一个异常，而不是返回一个错误消息，以符合C++的错误处理习惯。

  11. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  12. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  13. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  14. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  15. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  16. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  17. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  18. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  19. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  20. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  21. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  22. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  23. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  24. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  25. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  26. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  27. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  28. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  29. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  30. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  31. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  32. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  33. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  34. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  35. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  36. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  37. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  38. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  39. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  40. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  41. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  42. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  43. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  44. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  45. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  46. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  47. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  48. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  49. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  50. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  51. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  52. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  53. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  54. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  55. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  56. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  57. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  58. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  59. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  60. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  61. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  62. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  63. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  64. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  65. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  66. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  67. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  68. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  69. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  70. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  71. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  72. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  73. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  74. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  75. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  76. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  77. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  78. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  79. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  80. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  81. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  82. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  83. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  84. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  85. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  86. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  87. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  88. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  89. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  90. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  91. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  92. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  93. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  94. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  95. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  96. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  97. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  98. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  99. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  100. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  101. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  102. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  103. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  104. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

  105. 在NotificationManager::removePendingEntity函数中，removePendingEntity函数应该使用QHash::remove方法来移除元素，而不是使用迭代器。

106

@deepin-ci-robot
Copy link

deepin-ci-robot commented Dec 6, 2024

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 18202781743, yixinshark

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@18202781743
fix: default action can't work for notification
7b577a0 
Default action maybe have text, and it's actionId is default.
Remove entity when manually closing it before timeout.
CloseNotification can't work.

pms: BUG-290767
@18202781743
Copy link
Contributor Author

18202781743 commented Dec 6, 2024

/forcemerge

@deepin-bot
Copy link

deepin-bot bot commented Dec 6, 2024

This pr force merged! (status: blocked)

@deepin-bot deepin-bot bot merged commit 81ba22f into linuxdeepin:master Dec 6, 2024
7 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yixinshark yixinshark yixinshark approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@18202781743 @deepin-ci-robot @yixinshark